Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removed references to mcrypt #16062

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Removed references to mcrypt #16062

wants to merge 1 commit into from
+10 −469

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jan 13, 2025

mcrypt has been deprecated for quite some time. This removes the final vestigial references.

@what-the-diff What The Diff
Copy link

what-the-diff bot commented Jan 13, 2025

PR Summary

  • Removal of outdated 'mcrypt' references in the Docker setup
    The old and no longer supported 'mcrypt' commands have been removed from the Docker setup instructions.

  • Deletion of 'RecryptFromMcrypt' file
    The file responsible for re-encrypting data from the outdated 'mcrypt' format has been removed.

  • Removal of redundant encryption classes
    The outdated encryption classes (BaseEncrypter and McryptEncrypter) that utilized 'mcrypt' have been deleted.

  • Updating of 'snipeit.sh' script
    The shell script snipeit.sh has been updated to get rid of traces of 'php8.2-mcrypt', aligning with the removal of the 'mcrypt'.

@snipe snipe requested review from uberbrady and jerm January 13, 2025 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@uberbrady uberbrady Awaiting requested review from uberbrady

@jerm jerm Awaiting requested review from jerm

Assignees
No one assigned
Labels
backend cli / artisan docker
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@snipe